It's a bit unclear to me what version org.graalvm.version refers to.
This PR seems to assume it's the version of native image (including JIT compiler and truffle support).
But polyglot and truffle also seem to think it's their version although when using a bootstrap vm that wouldn't work.

@chumer, should PolyglotEngineImpl#getVersion stop using HomeFinder.getInstance().getVersion() and use META-INF/graalvm/org.graalvm.truffle/version or META-INF/graalvm/org.graalvm.polyglot/version instead?

@chumer, should PolyglotEngineImpl#getVersion stop using HomeFinder.getInstance().getVersion() and use META-INF/graalvm/org.graalvm.truffle/version or META-INF/graalvm/org.graalvm.polyglot/version instead?

Yes I think it should. We didn't get to cleaning this up yet. I also want to fully eliminate the HomeFinder. Happy to take contributions.

It's a bit unclear to me what version org.graalvm.version refers to.

From the name of property it seems rather clear it's the GraalVM version (legacy-style), as you say Native Image & Compiler version.
There is also the GRAALVM/release file which has e.g. GRAALVM_VERSION="25.0.1", which HomeFinder reads if the property is not already set, so that's further indication that the meaning of org.graalvm.version is the GraalVM version.

Could we merge this as a first step since it seems clearly a bug fix?

• For the case where GraalVM and Truffle versions match it doesn't matter.
• For the case they don't match, it reported the wrong version and now reports the right version.

should PolyglotEngineImpl#getVersion stop using HomeFinder.getInstance().getVersion() and use META-INF/graalvm/org.graalvm.truffle/version or META-INF/graalvm/org.graalvm.polyglot/version instead?

I can take a look at that, sounds easy enough.

I also want to fully eliminate the HomeFinder.

I'm not interested in doing that cleanup though.

I agree we can make truffle use its own version as a follow-up. I'll integrate this.